Skip to content

docs(core): Correct the documented default interpolation strategy - #86

Open
bhj wants to merge 1 commit into
cutterbl:masterfrom
bhj:fix/interpolation-strategy-default-docs
Open

docs(core): Correct the documented default interpolation strategy#86
bhj wants to merge 1 commit into
cutterbl:masterfrom
bhj:fix/interpolation-strategy-default-docs

Conversation

@bhj

@bhj bhj commented Jul 28, 2026

Copy link
Copy Markdown

Model used: Anthropic Opus 5

SoundTouchOptions.interpolationStrategy carried @defaultValue 'linear', but the default is lanczos: RateTransposer's constructor ends with setInterpolationStrategy(interpolationStrategy ?? 'lanczos'), and lanczos is the only strategy registered out of the box.

SoundTouchOptions.interpolationStrategy carried `@defaultValue 'linear'`,
but the default is `lanczos`: RateTransposer's constructor ends with
`setInterpolationStrategy(interpolationStrategy ?? 'lanczos')`, and lanczos
is the only strategy registered out of the box.

The wrong value is worse than cosmetic here. `linear` ships in
@soundtouchjs/interpolation-strategy-linear and has to be registered before
use, so following the doc and passing `'linear'` throws "Unknown
interpolation strategy id" from construction. The README already describes
the registry correctly and shows the registerLinearStrategy opt-in, so this
was the only place stating otherwise.

Adds a spec pinning the default, which nothing covered — the existing
strategy specs all pass an explicit id.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant